Skip to content

Adds auto-configuration of property useCodeAsDefaultMessage of MessageSource #10466

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

kedar-joshi
Copy link
Contributor

Fixes #10146 by adding auto configuration of useCodeAsDefaultMessage in MessageSourceAutoConfiguration

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Sep 30, 2017
Copy link
Member

@snicoll snicoll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry but it looks like that PR changed the formatting of the file.

Can you please rework the PR to use the same formatting rules so that only the things you've actually changed are included in the commit?

Also, please squash your changes to a single commit. You can push force on your existing branch to update this PR.

@kedar-joshi kedar-joshi closed this Oct 2, 2017
@snicoll snicoll removed the status: waiting-for-triage An issue we've not yet triaged label Oct 2, 2017
@kedar-joshi kedar-joshi reopened this Oct 2, 2017
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Oct 2, 2017
@kedar-joshi
Copy link
Contributor Author

@snicoll I apologies for the trouble. Line endings were changed to CRLF. I have recommitted the changes with proper formatting.

@snicoll snicoll self-assigned this Oct 2, 2017
@snicoll snicoll added priority: normal type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Oct 2, 2017
@snicoll snicoll added this to the 2.0.0.M5 milestone Oct 2, 2017
snicoll pushed a commit that referenced this pull request Oct 2, 2017
@snicoll snicoll closed this in 6c826ef Oct 2, 2017
snicoll added a commit that referenced this pull request Oct 2, 2017
* pr/10466:
  Polish
  Polish "Adds support for useCodeAsDefaultMessage"
  Adds support for useCodeAsDefaultMessage
@kedar-joshi
Copy link
Contributor Author

@snicoll Any plan to integrate this fix in 1.5.x ? I can backport it, if needed.

@snicoll
Copy link
Member

snicoll commented Oct 2, 2017

No, sorry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants